Full rewrite. Supports mouse, multimedia keys, auto-reconnect, etc. #11
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This was referenced Oct 11, 2023
Open
Use pythonic properties Add video links Include stacktrace
Include traceback
Use pathlib instead of os.path
Fix: Use logging.exception instead of .error
Cache attribute lookup
Add --list_devices argument
Update docs to reflect latest changes
Continue script when src dir empty
Add error messages on abort
Init the submodules
Update docs
|
Hi! There are some minor ones. I cannot report them in the fork repo, so here it goes:
Output if no device connected |
|
Okay, I've managed to workaround the second issue but the fix is really ugly. |
|
Hey @ig-sinicyn Thanks a lot for your feedback! I was aware of both these issues. In fact the second one already had a dedicated issue in my fork. Cool that you figured out a solution yourself right away! Feel free to PR your fix so this one may be closed. The first one got introduced when I switched from evdev PyPI package to my own submodule, since I wanted to be able to modify it. This one needs investigating why the submodule is not working properly. If you have a clue please let me know in this new issue!
Could you please explain? Do you mean my fork or yours? Cheers |
This was referenced Nov 14, 2023
|
Hi @quaxalber! :) I'm really glad the feedback was well-received. The bridge works with no issues, no reboots or hiccups so far. The only thing I've noticed, the state of NumLock/CapsLock is not synchronised with device the RaspberryPi is attached to. I've did not investigated it deeper so far so I'm not sure where the real problem is. It may be my device, raspberry, my keyboard or all of them:)
Well, it was my fault:) I've refreshed your's fork page and the issues tab appeared. I thought there's no issues for the forks, my bad. |
Make user owner of all files in repo
* Implemented grabbing input device * Chmod scripts executable * Bump version * Chmod main script executable * Use shell wrapper script that activates venv Add pip constraints * Chmod scripts * Remove dot imports * Add blank line * Modify log messages * Bump version * Update performs a clean reinstall * Fix executable path * Preserve repo ownership * Fix stat dir * Remove constraints * Re-enable constraints * Preserve currently checked out branch * Add cleanup function * Add cd to parent dir * Adapt reboot hint * Adapt to latest script changes * Remove unnecessary blank lines * Update to v0.6.6 * Add adafruit-circuitpython-typing * Set fixed version * Fix missing = * Reduce delays * Keep track of relayed devices * Add debug output * Initialize gadgets once and share across module * Move device functions to __init__.py * Move device functions back to relay.py * Remove temporary debug output Bump version * Defer USB device init * Check gadget state on relay init Rename symbols * Improve gadget init debug output * Update to v0.6.7 * Show versions before updating * Change to project dir first * Use variables instead of hard-coded paths * Make user owner of all files * Leave chmod to install.sh * Preserve ownership before installing * Bump version --------- Co-authored-by: Hannes U @cylonid Co-authored-by: Benjamin T @quaxalber
* Use asyncio.all_tasks() again to check if relay is active * Add missing type hints * Add Arguments class with typed properties * Reformat help texts * Unify punctuation * Print help arg last * Re-add print_help() * Rearrange help args * Update docs to v0.7.1 * Bump version
* Add Arguments.__str__() * Move arg setup to CustomArgumentParser
* Add Arguments.__str__() * Move arg setup to CustomArgumentParser * Fix object representations Fix type hints * Fix variable name * Fix relay str representation * Bump version * Run git fetch before tag * Log signal name instead of int on shutdown * Re-add sys.exit(0) in signal handler
* Make venv dir non-hidden
* Fix device representation * Remove str cast * Fix debug output * Bump version
* Update docs to v0.7.4
* Build Arguments.__str__() dynamically * Update docs
* Move intro to top * Specify RPi OS as mandatory requirement * Specify RPi OS Bookworm * Add link to RPi OS Bookworm blog post
* Clean up DeviceIdentifier * Add missing imports
* Set new overview image * Update overview image license
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hey @HeuristicPerson
Thanks so much for your project. It helped me getting started. Since I needed to relay both keyboard and mouse , I decided to build my own solution on top. Feel free to pull if you like it.
Cheers